Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(bazelrc-presets): restore 60s progress rate on CI #383

Merged
merged 1 commit into from
Mar 6, 2023
Merged

fix(bazelrc-presets): restore 60s progress rate on CI #383

merged 1 commit into from
Mar 6, 2023

Conversation

alexeagle
Copy link
Collaborator

Does this match RH Matt?

@alexeagle alexeagle enabled auto-merge (rebase) March 6, 2023 01:39
gregmagolan
gregmagolan approved these changes Mar 6, 2023
View reviewed changes
Copy link
Collaborator

@gregmagolan gregmagolan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

60s does seem like a better default for CI in general. Buildkite can handle 5s without being too spammy but GHA and CCI are not as nice.

@alexeagle alexeagle merged commit abaf464 into main Mar 6, 2023
@alexeagle alexeagle deleted the docs branch March 6, 2023 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gregmagolan gregmagolan gregmagolan approved these changes

@mattem mattem Awaiting requested review from mattem

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alexeagle @gregmagolan